Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i18n: Weblate updates (fixed merge errors) #2020

Merged
merged 6 commits into from
Dec 5, 2023
Merged

i18n: Weblate updates (fixed merge errors) #2020

merged 6 commits into from
Dec 5, 2023

Conversation

acolomb
Copy link
Member

@acolomb acolomb commented Dec 5, 2023

Unfortunately, because of recent troubles with Weblate's internal Git repository, some changes were lost on the pushed branch. This current state now contains all changes up-to-date.

Please merge soonish so that Weblate can pick up the updated upstream branch and continue working normally.

acolomb and others added 6 commits December 5, 2023 10:21
Currently translated at 100.0% (369 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/de/
Currently translated at 100.0% (369 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/pl/
Currently translated at 100.0% (369 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/pl/
Currently translated at 97.2% (359 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/cs/
Currently translated at 100.0% (369 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/pl/
Currently translated at 100.0% (369 of 369 strings)

Translation: Syncthing/Android app strings
Translate-URL: https://hosted.weblate.org/projects/syncthing/android/stringsxml/tr/
@acolomb acolomb marked this pull request as ready for review December 5, 2023 09:25
@imsodin imsodin merged commit f071fa7 into main Dec 5, 2023
1 check passed
@imsodin
Copy link
Member

imsodin commented Dec 5, 2023

I hope merging (well fast-forward here) was the right thing to do, and it didn't need a squash. If you have permissions on github, I'd be ok with you fixing up weblate issues directly as well. If you don't have access to do that and think it would help, let me know and I'll see what I need to do to get you the necessary permissions.

@acolomb
Copy link
Member Author

acolomb commented Dec 5, 2023

Oh dear... Yes, it would really help to have direct push access to the main branch here (which I don't have yet). Problem is that new translations are coming in continuously, so whenever I try to push merge conflicts into the main branch for Weblate to finally reach a conflict-free state, they often end up causing new conflicts before the PR is reviewed and merged. All because of an internal Weblate bug which I reported and their attempt to fix it on our repository actually discarded some changes.

I hope when we do reach a clean state again, such troubles will not occur during normal interaction with Weblate.

@acolomb
Copy link
Member Author

acolomb commented Dec 5, 2023

In the meantime, if you get to it first: Merge the weblate branch into main again, there are two more translation commits by now.

@imsodin
Copy link
Member

imsodin commented Dec 5, 2023

You should now have access and I just merged weblate again.

@acolomb
Copy link
Member Author

acolomb commented Dec 5, 2023

Verified working, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants